Add sans-fitter to conda-forge - #34564
Conversation
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
|
@conda-forge/help-python ready for review! |
Co-authored-by: UENO, M. <eunosium-1128@live.jp>
| - python: | ||
| imports: | ||
| - sans_fitter | ||
| pip_check: false # sasmodels 1.1.0 has unresolved tccbox metadata |
There was a problem hiding this comment.
We need to fix this in statsmodel first.
There was a problem hiding this comment.
Thanks for the review @ocefpaf! I'm trying to get this fixed upstream first SasView/sasmodels#752 but it will probably take some time going through the stack and a release. Is it possible to get it on conda-forge and then I can update the feedstock with the pip check with the new release?
PS: no 't', sasmodels :D
There was a problem hiding this comment.
See conda-forge/sasmodels-feedstock#20 for the fix. Can you mark the build number 0 as broken?
There was a problem hiding this comment.
There was a problem hiding this comment.
Updated this recipe to do a pip check and it's happy, thanks!
Checklist
recipe.yamlformat, or this PR explains the exceptional requirement for deprecated v0.url) rather than a repo (e.g.git_url) is used in your recipe (see here for more details).